Can't return values from functions that don't return anything
authorReedy <reedy@wikimedia.org>
Thu, 19 Apr 2012 13:07:40 +0000 (14:07 +0100)
committerReedy <reedy@wikimedia.org>
Thu, 19 Apr 2012 13:07:40 +0000 (14:07 +0100)
Change-Id: If964ce2f39904f1e9e3101e7cafaa6a96d58adfc

includes/api/ApiQueryBase.php
includes/db/ORMRow.php
includes/upload/UploadFromStash.php
includes/upload/UploadFromUrl.php

index 2666045..a633748 100644 (file)
@@ -233,7 +233,7 @@ abstract class ApiQueryBase extends ApiBase {
         */
        protected function addTimestampWhereRange( $field, $dir, $start, $end, $sort = true ) {
                $db = $this->getDb();
-               return $this->addWhereRange( $field, $dir,
+               $this->addWhereRange( $field, $dir,
                        $db->timestampOrNull( $start ), $db->timestampOrNull( $end ), $sort );
        }
 
index a063f72..4ac41cc 100644 (file)
@@ -207,7 +207,7 @@ abstract class ORMRow {
         * @param integer|null $id
         */
        public function setId( $id ) {
-               return $this->setField( 'id', $id );
+               $this->setField( 'id', $id );
        }
 
        /**
index 8805c50..317dd55 100644 (file)
@@ -99,7 +99,7 @@ class UploadFromStash extends UploadBase {
                // chooses one of wpDestFile, wpUploadFile, filename in that order.
                $desiredDestName = $request->getText( 'wpDestFile', $request->getText( 'wpUploadFile', $request->getText( 'filename' ) ) );
 
-               return $this->initialize( $fileKey, $desiredDestName );
+               $this->initialize( $fileKey, $desiredDestName );
        }
 
        /**
index 0ae8a22..49bfb91 100644 (file)
@@ -93,7 +93,7 @@ class UploadFromUrl extends UploadBase {
                if ( !$desiredDestName ) {
                        $desiredDestName = $request->getText( 'wpUploadFileURL' );
                }
-               return $this->initialize(
+               $this->initialize(
                        $desiredDestName,
                        trim( $request->getVal( 'wpUploadFileURL' ) ),
                        false